Skip to content

Conversation

@Pyasma
Copy link
Contributor

@Pyasma Pyasma commented Dec 29, 2025

Added function sql_alchemy_uri
Added Tests
Run prek tests
#38195


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@potiuk potiuk force-pushed the pyasma/Fix-Apache_HiveServer2Hook branch from 180ffd6 to 668b937 Compare December 29, 2025 18:47
@potiuk
Copy link
Member

potiuk commented Dec 29, 2025

Rebased - I think there were issues with github APIS - mocked in one of the recent commits.

@potiuk
Copy link
Member

potiuk commented Dec 29, 2025

It looks good, one thing I noticed is that we miss sqlalchemy dependencies and handling lack of sqlalchemy installed for our providers that have "some" sqlalchemy functionality. I created #59895 as good first issues to fix them in all providers.

@potiuk potiuk changed the title Pyasma/fix apache hive server2 hook Fix apache hive server2 hook Dec 29, 2025
@potiuk potiuk changed the title Fix apache hive server2 hook Fix apache hive server2 hook for sqlalchemy URL Dec 29, 2025
@potiuk potiuk merged commit 4670fc0 into apache:main Dec 29, 2025
87 checks passed
@Pyasma
Copy link
Contributor Author

Pyasma commented Dec 30, 2025

Thanks @potiuk

Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
* feat: introduce `sqlalchemy_url` property and `get_url` method to Hive hook

* pushing the tests
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
* feat: introduce `sqlalchemy_url` property and `get_url` method to Hive hook

* pushing the tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants